Skip to content

Drop support for Node.js 18#499

Merged
remcohaszing merged 1 commit intomainfrom
drop-node-18
Apr 13, 2025
Merged

Drop support for Node.js 18#499
remcohaszing merged 1 commit intomainfrom
drop-node-18

Conversation

@remcohaszing
Copy link
Copy Markdown
Member

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and discussions and couldn’t find anything or linked relevant results below
  • I made sure the docs are up to date
  • I included tests (or that’s not needed)

Description of changes

The asynchronous TypeScript plugin keeps causing issues. The goal is to move away from that and make it synchronous. Node.js 20.19 adds support for require(esm), which is a great first step for our use case.

VSCode currently ships with Node.js 20.18. However, this is still fine, as dependencies are bundled in the VSCode extension.

The asynchronous TypeScript plugin keeps causing issues. The goal is to
move away from that and make it synchronous. Node.js 20.19 adds support
for `require(esm)`, which is a great first step for our use case.

VSCode currently ships with Node.js 20.18. However, this is still fine,
as dependencies are bundled in the VSCode extension.
@remcohaszing remcohaszing added 🦋 type/enhancement This is great to have 🏡 area/internal This affects the hidden internals 🧑 semver/major This is a change 🤞 phase/open Post is being triaged manually labels Apr 13, 2025
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 13, 2025

🦋 Changeset detected

Latest commit: 09852e9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@mdx-js/typescript-plugin Minor
@mdx-js/language-service Minor
@mdx-js/language-server Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.02%. Comparing base (c589e22) to head (09852e9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #499   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files          10       10           
  Lines        1769     1769           
=======================================
  Hits         1628     1628           
  Misses        141      141           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@remcohaszing remcohaszing merged commit d440c79 into main Apr 13, 2025
19 checks passed
@remcohaszing remcohaszing deleted the drop-node-18 branch April 13, 2025 12:43
@github-actions

This comment has been minimized.

@github-actions github-actions Bot mentioned this pull request Apr 13, 2025
@remcohaszing remcohaszing added the 💪 phase/solved Post is done label Apr 13, 2025
@github-actions github-actions Bot removed the 🤞 phase/open Post is being triaged manually label Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏡 area/internal This affects the hidden internals 💪 phase/solved Post is done 🧑 semver/major This is a change 🦋 type/enhancement This is great to have

Development

Successfully merging this pull request may close these issues.

1 participant